Skip to content

feat: add share and import features#340

Merged
aldbr merged 2 commits into
DIRACGrid:mainfrom
TheauW:twartel-share-app-state
May 13, 2025
Merged

feat: add share and import features#340
aldbr merged 2 commits into
DIRACGrid:mainfrom
TheauW:twartel-share-app-state

Conversation

@TheauW
Copy link
Copy Markdown
Contributor

@TheauW TheauW commented May 9, 2025

This PR introduces two new buttons that allows to easily share and import application state. These features enable simple and quick exchange of selected applications between users.

  • The share button generates a sharable representations of the current application state, which can be copied and sent to others.
  • The import button allows users to load a previously shared state, restoring the same application configuration

Copy link
Copy Markdown
Contributor

@aldbr aldbr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks great! Thank you very much 🙂

Could you add some documentation for the developers to explain how to create a state in an "Application" component (e.g. JobMonitor) please?

  • As a developer, do we need to implement exportState and setState in an application? (it will be very similar to every applications, am I wrong? Would it make sense to move these functions elsewhere?)
  • How am I supposed to save the state of my application? You can explain the session storage and the <app_id>_State, and how you add/remove details from there (you can use simple examples to highlight the concepts).

Thanks again!

@TheauW TheauW force-pushed the twartel-share-app-state branch from a504a35 to cd9ee26 Compare May 13, 2025 09:36
Comment thread docs/developer/create_application.md Outdated
Comment thread docs/developer/create_application.md Outdated
@TheauW TheauW force-pushed the twartel-share-app-state branch from cd9ee26 to 502f543 Compare May 13, 2025 14:48
@aldbr aldbr merged commit 51d243c into DIRACGrid:main May 13, 2025
14 checks passed
@TheauW TheauW deleted the twartel-share-app-state branch May 13, 2025 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants